Skip to content
This repository was archived by the owner on Apr 6, 2023. It is now read-only.

fix(nuxt): replace process.dev in nitro bundle#5852

Merged
pi0 merged 3 commits intomainfrom
fix/vite-node-dev
Jul 12, 2022
Merged

fix(nuxt): replace process.dev in nitro bundle#5852
pi0 merged 3 commits intomainfrom
fix/vite-node-dev

Conversation

@antfu
Copy link
Copy Markdown
Member

@antfu antfu commented Jul 12, 2022

process.dev is undefined in the npm build (while it works in the local playground). Since NUXT_VITE_NODE_OPTIONS will only be injected in dev, I guess it's safe to remove it

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 12, 2022

Deploy Preview for nuxt3-docs canceled.

Name Link
🔨 Latest commit 245a39b
🔍 Latest deploy log https://app.netlify.com/sites/nuxt3-docs/deploys/62cd688992732b000976f354

@antfu antfu changed the title fix(vite-node): process.dev undefined in production causing FOUC fix(vite-node): process.dev undefined in npm release causing FOUC Jul 12, 2022
@antfu antfu changed the title fix(vite-node): process.dev undefined in npm release causing FOUC fix(core): replace process.dev in Nitro Jul 12, 2022
@pi0 pi0 changed the title fix(core): replace process.dev in Nitro fix(nuxt): replace process.dev in nitro bundle Jul 12, 2022
@pi0 pi0 merged commit 135dfdf into main Jul 12, 2022
@pi0 pi0 deleted the fix/vite-node-dev branch July 12, 2022 12:32
@pi0 pi0 mentioned this pull request Jul 12, 2022
@danielroe danielroe added the 3.x label Jan 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants